辞書画面の機能追加（ソート、検索、リスト機能強化） #2411
Conversation
|
レビューお待たせしてしまってすみません！ |
見方があっているか確認ではあるのですが、一緒にエラー出てる117行目と119行目の内容の話であっていますでしょうか？
|
|
@kebin628 原因の特定までできてないのですが、おそらくこのPRの変更でエラーになったんだと思います！ |
|
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ： |
すみません、該当部分でなく、エラーの本体というかきっかけとしてはv-forの返り値が原因だったようです。 それと、Mainマージしたらこっち側でエラーが出るようになってしまったため、それの対応も行っています。 読み方＆アクセント辞書ダイアログの右側パネルを別コンポーネントにする by jdkfx · Pull Request #2290 · VOICEVOX/voicevox 対応内容としては、 よろしくお願いします。 |
There was a problem hiding this comment.
すみません、レビューがとても遅くなりました！！！
あれからnpmがpnpmに変わったりしました！
#2512
UIの相談をさせていただきたく･･･ 🙇
ソートUIに関して、issueの提案のようにメニューのボタンにするの良さそうな気がしてたのですがどうでしょうか 👀
別レーンにするのは･･･････横幅が足りないから的な感じですかね？だとしたらさてどう解決したものかって感じですねぇ 😇
こういうUIも良さそうな気がしました。。メニューで「並び方」と「昇順・降順」を切り替えられる感じを想像してます。
ただ今どういう並び順なのかの表示がちょっと難しい気もするのですが。。。。
良い解決策が出てない中でのコメントですみません。。。 🙇
|
@Hiroshiba 読み順の昇順降順別表記内部で昇順降順が分離している件については纏める事はできますが、これ今後要素増えたりしないですかね？ とはいえ、今回の場合対象は現状読みと優先度くらいですし、それ以上増える要素もなさそうなので、ここは指摘の通り、対象と昇順降順をマージしてやって
今どれにしているかの話については、SelectUIの上に現在の選択状態がそもそも出るので、「↑↓並び替える」でなく、「↑↓読み順: 昇順」で設定内容出すようにしてやれば解決するかなとは（＝「並び替える」の文字列は、設定中の値とアイコンから逆読みできるので不要なのでは）
メニュー欄の内容
幅が割と限界ギリギリですね･･･ 読み順がどれに設定されているのかを表示しなくて良いのであれば、アイコンだけで完結できます。
ひとまず、最後のやつが見栄え的には割とスッキリしてそうなので、この方向も良さそうかなと仮組みしてて感じてました。 余談手元の他ソフトでどうしてるか参考資料として置きます。 VOICEPEAK A.I.VOICE2 CeVIO AI ひとまず詰め切るまで手は動かさないでおきます～ |
内容
辞書画面の機能を追加しました。
※ Issueにある提案機能の内、インポートエクスポート以外を実装した形になります。
関連 Issue
【提案】読み方＆アクセント辞書の機能追加 · Issue #2381 · VOICEVOX/voicevox
#2381
スクリーンショット・動画など
読み順
優先度順
サブメニュー（優先度切り替え。今後インポート・エクスポートもここに入れる）
検索機能
その他
ソートのサイズの都合、当初案のUIから多少変更になっています。
リストでの優先度表示はアプリを消しても設定が消えないようにしています。
設定画面で入れるほど大掛かりなものでないので、一時的にローカルストレージに保存できるようになっていればいいなと思って入れていますが、他に良さそうなやり方があればそちらを使います。（OnClickShowPriorityOnDictionary）